Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve initial Prompt #274

Merged
merged 1 commit into from
Jul 13, 2022
Merged

Improve initial Prompt #274

merged 1 commit into from
Jul 13, 2022

Conversation

DonRichards
Copy link
Member

@DonRichards DonRichards commented Jul 13, 2022

On a fresh install, there are several prompts. This is a bit confusing and doesn't lay out all of the options for generating secrets.

The new prompt looks like this

$ make up

Checking secrets...
  No secrets found in /home/user/isle-dc/secrets/live/

Everything (Drupal, SQL, Solr, etc.) needs passwords to run. These passwords are stored in the secrets directory.
  1. Generate new secrets
       ╰ This will generate random new passwords and output them into /home/user/isle-dc/secrets/live/
  2. Copy the existing secrets
       ╰ This will copy the existing secrets into the /home/user/isle-dc/secrets/live/. Every password will likely be 'password'.
  3. Create your own secrets
       ╰ This will copy the secrets from /home/user/isle-dc/secrets/template/ to /home/user/isle-dc/secrets/live/
            and will exit so you can edit the secrets manually prior to building the Docker image.
  4. Exit

Enter your choice: 3
Creating your own secrets. Copying templates to live directory.

To test:

  • Try each option. Starting in reverse order to speed up testing.
  • Run make clean before each test so that the secrets/live/ directory is empty.
  • Options # 3 and # 4 should exit the build process entirely when complete.
  • Final test - without running make clean run make down then make up. It should completely bypass the above prompt.
  • Options # 1 & # 2 should build completely and you should be able to log into Drupal with the password in secrets/live/DRUPAL_DEFAULT_ACCOUNT_PASSWORD

@rosiel rosiel merged commit b62b1ed into development Jul 13, 2022
@rosiel rosiel deleted the generate_secrets_prompt_impr branch July 13, 2022 16:27
@rosiel rosiel mentioned this pull request Jul 13, 2022
@DonRichards
Copy link
Member Author

Thanks @rosiel

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants